Skip to content

Conversation

@tariq1890
Copy link
Contributor

No description provided.

@coveralls
Copy link

coveralls commented Jan 14, 2026

Coverage Status

coverage: 25.669% (+0.02%) from 25.65%
when pulling 2ce61f9 on rocky-support
into 662466c on main.

@tariq1890 tariq1890 force-pushed the rocky-support branch 3 times, most recently from b7f8fee to 28df39e Compare January 28, 2026 18:46
@tariq1890 tariq1890 marked this pull request as ready for review January 28, 2026 18:46
func (n nodePool) getOS() string {
if n.osRelease == "rocky" {
// If the OS is RockyLinux, we will omit the RockyLinux minor version when constructing the os image tag
n.osVersion = strings.Split(n.osRelease, ".")[0]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be

Suggested change
n.osVersion = strings.Split(n.osRelease, ".")[0]
n.osVersion = strings.Split(n.osVersion, ".")[0]

?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, great catch! Thanks

Signed-off-by: Tariq Ibrahim <tibrahim@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants